Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Change how grids save state on navigation #842

Open
wants to merge 1 commit into
base: vnext
Choose a base branch
from
Open

Change how grids save state on navigation #842

wants to merge 1 commit into from
+33 −12

Conversation

mddragnev
Copy link
Member

@mddragnev mddragnev added the status: in-review PR is ready for a review label Nov 12, 2024
@mddragnev mddragnev mentioned this pull request Nov 12, 2024
Open
@dkamburov dkamburov requested a review from skrustev November 14, 2024 13:48
@tishko0 tishko0 added status: verified PR is ready for merging and removed status: in-review PR is ready for a review labels Nov 19, 2024
@skrustev
Copy link
Member

skrustev commented Jan 7, 2025

Not sure what was the reason for the change here? I see in the issue you discuss another way of handling it using the LocationChanging event but the current implementation also work? It does seem to be a newer way of handling navigation, but is that just it?

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@skrustev skrustev Awaiting requested review from skrustev

At least 1 approving review is required to merge this pull request.

Assignees

@dkamburov dkamburov

@tishko0 tishko0

Labels
status: verified PR is ready for merging
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mddragnev @skrustev @dkamburov @tishko0